Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

frontend: Work On The Error Page This commit is for mo… #34

Closed
wants to merge 2 commits into from

Conversation

asmit27rai
Copy link
Contributor

@asmit27rai asmit27rai commented Jun 1, 2024

Design Error Page

Fixes: #21

Description

I used tailwind css to make this error page which render when known route get accessed and has go to homepage option created by link of react-router-dom with some modification using tailwindcss.

How to Test

Just Run the "npm run dev" command in frontend and on app page try to access page of unknown route.

Checklist

  • I have tested these changes locally.
  • I have reviewed the code and ensured it follows the project's coding guidelines.
  • I have updated the documentation, if necessary.
  • I have assigned reviewers to this pull request.

Screenshot

Contri_2

@criticic
Copy link
Contributor

criticic commented Jun 1, 2024

I think you should add a screenshot in the PR description of the page added.

@kuv2707
Copy link
Collaborator

kuv2707 commented Jun 1, 2024

@asmit27rai Please make the commit heading shorter and leave a line before writing the body. That way, it renders well.

… pages that show if unknown route is accessed and then it give link to homepage by using link from react-router-dom and also remove display flex option from body \n Fixes: shivansh-bhatnagar18#21
This commit is for modified error pages that show if unknown route is accessed and then it give link to homepage by using link from react-router-dom and also remove display flex option from body

Fixes: shivansh-bhatnagar18#21
@shivansh-bhatnagar18
Copy link
Owner

Thank you @asmit27rai for your Pull Request. Every Pull request matters but we have found out a better Error page for the same. For now I am closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Design an error page
4 participants